Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Copy Value as Text to the inspector #75604

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KoBeWi
Copy link
Member

@KoBeWi KoBeWi commented Apr 2, 2023

Editor properties have Copy Value context option, but it copies the value as Variant into some internal inspector clipboard. I thought it would be useful if there was a way to copy that value as text (as I tried to copy a position from remote node).

godot.windows.editor.dev.x86_64_zeg9lewjZk.mp4

"Supersedes" part of #75274 (which adds a similar option)
Doesn't work well with Objects.

Alternatively we could make Copy Value copy both to the inspector clipboard and the text one.

@kleonc
Copy link
Member

kleonc commented Apr 2, 2023

Alternatively we could make Copy Value copy both to the inspector clipboard and the text one.

Alternative PR: #71275. 🙃

@stebulba
Copy link
Contributor

stebulba commented Apr 2, 2023

Part of this proposal godotengine/godot-proposals#2951

@akien-mga
Copy link
Member

Alternatively we could make Copy Value copy both to the inspector clipboard and the text one.

Alternative PR: #71275.

I think this might be the better / simpler option.

We just need to figure out what to do in cases where the construct string isn't useful, like Object/Resource.

@Zireael07
Copy link
Contributor

I am one of the people who are in favor of either (I often bemoaned the fact that I had to type the value over in the script window just because it was in some other scene) and I have no preference towards either solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants